-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ExtrusionMenu-Load/UnLoadToMoreMenu #1466
ExtrusionMenu-Load/UnLoadToMoreMenu #1466
Conversation
Sync with original
Sync 16.XII.2020
Sync 2020.XII.19
Sync 24.XII.2020
…ouchScreenFirmware # Conflicts: # TFT/src/User/API/SpeedControl.c
…ouchScreenFirmware into Menu-More---Load/Unload
Sync 31.XII.2020
…into Menu-More---Load/Unload
There's now a GitHub action that will automatically compile & replace existing firmware binaries after a PR is merged or changes are committed directly to the master branch, so those are no longer needed and can be removed. |
My action didn't previously check what repo it was generating binaries for, just the master branch. |
The PR source wasn't from |
I never work on the master branch, I always create a new branch for every idea, fix, etc. The bin files were generated automatically by VSCode. |
kisslorand I guess you are not using the gcode file preview and therefore you do not care even a little bit how this affects other users which are using this function. But you can not change the firmware the way you would like it, without taking a bit care of others. Why you are not making a private version, where you can do whatever you want? |
@oldman4U Later edit: somehow I had a feeling you will jump at my throat with this PR, I was very eager what would you find this time |
You added great things to the firmware and you fixed some painful issues, nobody wanted to fix. But you also removed existing features, just because you did not need it and you wanted to force your own solution. So in addition to the Back button which opens the same window you get when you press the Print button on the main screen, there is now also a button labeled BabyStep, showing the preview of the printed element. I wish you would see the benefit of working together, instead of doing what you believe is right, because it works for you. |
@oldman4U is this an in progress design? |
The design is available for all pages of the TFT35 resolution using 480x320. From this point on it is math to adjust it for other resolutions. What's missing from my side are the buttons itself for all resolutions. Gurmeet started with the implementation but has no time anymore. The first step is to remove the echo window from the main screen and to add the lower bar. To be able to remove the echo window he added the ability to see the last three echo notifications in an extra window which is the button in the lower left corner. This is where it stopped.;-( |
I am glad to hear your health is in a good shape and you are sober. No one asked about your health nor nobody said that you are drunk but it's nice that you shared with us your well-being. (Please do not put words in my mouth I never said, it's disgusting.) I am asking again, what is wrong with this PR? Please provide that detail so I can fix it.
What is this "respectful level" all about? What are you even talking about? It is addressed directly to me, accusing me about something that I do not have the slightest clue. Would you please show us how respectful is your level and explain what have I done in this PR and how do I "do not care even a little bit how this affects other users which are using this function"? Babystep, file preview with babystep label, Back were there during print since I first installed this FW to my TFT. Babysteps and More were removed in a later PR but only AFTER print is finished. As @mehmetsutas mentioned:
Yeah, you haven't tested it but it must be wrong because it is made by me. Your sought respect is bursting with abundance. In case you missed (again) my request, please explain what is wrong in this PR so I can fix it. Please let me know the awful thing done in this PR that led you to cover me with such an amount of filth. |
@kisslorand please on extrude.c apply your changes starting from my file in my PR. I changed some indentation. @oldman4U Overall it seems to me this PR simply makes the load/unload menu available during a print without sacrifacing other features. So it is should be ok. If babystep or other features were removed, I will be the first one asking to revert the changes |
Yes it is. Good |
So, after the circus where @oldman4U "respectfully" showed us his true colors by his pointless piss contest out of the blue, things can move on to some beautiful prints. If no one else (than @You_Know_Who) finds any problem with this PR, it can be merged peacefully. P.S.: NO, the things introduced in this PR are NOT user selectable, there is NO adjustable thing in config.ini nor in parameters settings. There's NO popup, there's NO info that can/could/should be seen from "a certain distance". Nothing was removed, it's an extra that you can use if you wish or not, that's the only option. |
Following the advice of a good friend, I will not stretch this even further. But maybe others want to know what kind of person you are. Regarding ill-founded. Please see #1359 @digant73 Thank you for checking. It is all good. |
Would you care to explain? How is mentioning alcohol a lack of respect?
Good excuse to not explain what is your issue with this PR, women style.
Would you care to explain? I cannot find anything similar to what you did. |
So you are also a sexist.... |
Yes, I prefer women. :) The truth is I love them . |
For all of you reading this. I am really sorry |
Yeah, you should!
You are very active but still failing to explain what is your issue with this PR. This can make people wonder. |
i hope when btt gets around to reviewing this after they accept/reject the pr the comments are locked |
It will not stop Herr You_Know_Who to spit on me in the future. |
This is ready to be merged, no bugs related to this PR were reported nor found. |
Requirements
BTT TFT or MKS TFT
Description
Related Issues
Fixes #1423
RAM & Flash impact
RAM ->0 bytes extra
Flash -> 304 bytes extra